Skip to content

Updated SAS weighted logrank test page - #625

Closed
dimpz4295 wants to merge 5 commits into
PSIAIMS:mainfrom
dimpz4295:qmd_updates
Closed

dimpz4295 wants to merge 5 commits into
PSIAIMS:mainfrom
dimpz4295:qmd_updates

Conversation

@dimpz4295

Copy link
Copy Markdown

Updated the SAS weighted log-rank test page to include table with numerical outputs from the SAS procedures using sample data.

@abiterry

abiterry commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

Hi @dimpz4295, thanks so much for contributing these two PRs! Before we review the content, please could you pull down from the main CAMIS branch into your forked branch and add library(tibble) to your package calls for both PRs?

@dimpz4295

Copy link
Copy Markdown
Author

Hi @abiterry yes I can! I apologize for that. I'll do that as soon as I can and fix the errors.

@dimpz4295

dimpz4295 commented Oct 11, 2026 •

Copy link
Copy Markdown
Author

Hi @abiterry! I've pulled the latest changes from the main CAMIS branch into my ancova-fix branch and 'weighted log rank SAS page' branch, added library(tibble) to the package calls, and pushed the updates to my fork. Thank you.

Comment thread Comp/r-sas_ancova.qmd
@DrLynTaylor

Copy link
Copy Markdown
Collaborator

Hi @dimpz4295 - just saw you pushed ANCOVA in a separate PR that now runs clean. Can you just remove ANCOVA change from this PR, then we can bring them both in separately. Thanks !

@DrLynTaylor

Copy link
Copy Markdown
Collaborator

Thanks @dimpz4295 - This has been pulled in through a new PR #627 because I think if I pull in this PR, ANCOVA will be changed back to the original (before your PR#626)! You should be able to see the changes live on the repo by latest tomorrow. Do let @abiterry and me know if any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants